Don't #include debug

A near rewrite of the reader of the stmsdf reader to more effectively parse fields as QDate,Times,Floats, etc. Still not very readable, but should be more bulletproof for bad input.

Don't name params in decls in itracku

Sign extension fix in EasyGPS

Remove xstrrstr and BASE_STRUCT from defs.h

Remove write-only variable in compegps.

Drop some unused method protos from CET.

XmlStreamWriter: make single arg methods explicit. Drop unused methods.

add cppcheck target to GPSBabel.pro. (#280)

* add cppcheck target to GPSBabel.pro.

fix a few cppcheck errors.

* fix a few cppcheck items.

strptime fix is from upstream.

generate html coverage reports. (#281)

* generate html coverage reports.

* tweak coverage recipe.

remove obsolete msvc workarounds. (#278)

msvc 2012 and earlier are not supported.
They didn't support c++14 anyway which we currently require.

use QThread::usleep instead of our own function. (#277)

Our test suite doesn't test sleep, but the nmea pause option
can be used on write to verify functionality interactively.

obsolete DEBUG_MEM (#279)

simplify conversion from utm. (#275)

* simplify conversion from utm.

add test cases for Norway and Svalbard exceptions.

* Correct bug in computing latitude band X.

enhance test to cover exceptional width of band X.

update shapelib to 1.4.1 (#274)

enhance sort filter for rtes & trks. (#273)

use bool, true, false. (#272)

add msvc2017 builds to appveyor. (#270)

update docker build script to use latest bionic. (#269)

Merge pull request #266 from tsteven4/clazy1

Incorporate improvements found by Qt oriented code checker clazy

Merge pull request #267 from gpsbabel/gpsbabel-patch-2

Remove leftover code from 'adding class' change

Remove leftover code from 'adding class' change

As tsteven4 points out, No reasing to move is_internal to local and back

fix clazy detected 'use isEmpty() instead'

this is from -Wclazy-isempty-vs-count

fix clazy detected 'Use *Ref() instead'

These are from -Wclazy-qstring-ref, and were automatically fixed.

fix clazy detected 'unused *'

these are from -Wclazy-unused-non-trivial-variable

fix clazy detected 'allocating an unneeded temporary container'

these are from -Wclazy-container-anti-pattern

Also, consistently use range based for loops in csv_util.cc

fix clazy detected 'c++11 range-loop might detach Qt container'

these are from -Wclazy-range-loop

fix clazy detected 'Use multi-arg instead'

these are from -Wclazy-qstring-arg

update travis for xenial (#265)

* update travis to use xenial.

add new travis target to build on xenial.
switch travis coverage generation to xenial.
update codacy upload tool for compatibility with newer java on xenial.

* fix gcov issues with "cannot open notes file"

support kml 2.3 track element. (#264)

* support kml 2.3 track element.

relax ordering requirement for gx:Track when and
coord children.

* fix kml includes and indentation.

Merge pull request #261 from gpsbabel/gpsbabel-patch-2

Update dependencies, notably for <optional>

Replace queues with Qlist in session handling. (#263)

session tracks a possible series of input formats and input filenames.

Merge pull request #262 from tsteven4/tcache

extend travis mac cache timeout.

extend travis mac cache timeout.

Merge pull request #260 from tsteven4/cpp14_msvc

move to c++14 standard.

Merge pull request #259 from tsteven4/queue_bye_bye

eliminate queues in csv style handling.

Update dependencies, notably for <optional>

Merge pull request #256 from tsteven4/rpath

have configure conditionally set rpath.

move to c++14 standard.

eliminate queues in csv style handling.

also:
fix bug with style option encoding.
pass some parameters by const reference.
eliminate some unused time fiddling.

Merge pull request #258 from tsteven4/sortbug

correct sort filter time option to use msecs.

correct sort filter time option to use msecs.

use consistent whitespace in configure comments.

have configure conditionally set rpath.

this may be necessary when using a Qt installation
that was not provided by the system.

Merge pull request #255 from gpsbabel/parallel

Parallelize CLI and GUI make.

Parallelize CLI and GUI make. testo and vtesto are harder.

Merge pull request #254 from gpsbabel/tweaks

Move more xcsv processing into QStrings. Fewer conversions

Allow string to be modified during parse. Ugh.

Let xcsv parser keep QStrings up for one more level.

Push QStrings -> char * conversions down one more level.

Merge pull request #253 from gpsbabel/tweaks

Find gsed when doing a tools/docfix on a Mac populated by Homebrew: G…

Find gsed when doing a tools/docfix on a Mac populated by Homebrew: GNU sed is /usr/local/bin/

Merge pull request #252 from gpsbabel/tweaks

Find gsed when doing a tools/docfix on a Mac populated by Homebrew: GNU sed is /usr/local/bin/

Merge pull request #249 from codacy-badger/codacy-badge

Add a Codacy badge to README.md

Add codacy badges to Github README.md

Add Codacy badge

Merge pull request #246 from tsteven4/trackdata

introduce optional class

introduce optional class

for things that may or may not have a value.
demonstrate usage with computed_trkdata.

Merge pull request #245 from tsteven4/exif2

a few exif cleanups.

a few exif cleanups.

Merge pull request #230 from tsteven4/bng

allow unicsv bng input to use all numeric grid references.

tweak unicsv bng documentation thanks Hornbydd.

Merge pull request #238 from tsteven4/exif

update exif format replacing queues, legacy time_t, tm, localtime, gmtime with Qt classes.
enhance exif rational number approximation using continued fractions.
support uncompressed thumbnails on write.
make debug support dynamic based on debug level instead of a conditional compile.
enhance exif test for write.

add exif support for uncompressed thumbnails.

fix exif dynamic debug omissions.

fix exif warnings.

replace legacy time handling in exif.

also add support of exif OffsetTime* tags.

minor fixes for exif.

use QVariant to store exif tag data.

enhance exif debug capability.

use continued fractions to create TIFF RATIONAL types.

some references were regenerated as the new algorithm
can generate different values for the RATIONAL numerator
and denominator, in this case 0.0 -> 0/INT32_MAX.

improvements to exif format.

fix bugs with rational types including greatest common divisor.
turn off assertions for release builds in GPSBabel.pro.
clean up exif.

tweak endian routines for consistency. (#242)

align util routeines with prototypes using readability-inconsistent-declaration-parameter-name.

eliminate queues in inifile. (#239)

* eliminate queues in inifile replacing them with QHashs

* replace use of gbfile in inifile with QTextStream

Final presubmit cleanups

Ship it when it’s green. (and it’s tomorrow, errr, today.)

Fix leak in last cleanup.

Mixing C pointers and sane pointers is killing me…

Rewrite xcsv_parse_style_buff() in a readable way

…might be more clever with a QStringRef, but this is hardly a
performance path.

Degrubbify for submit

Remove “global” char*’s in body of csv format parser. Make argument
type String to allow fixing callers later. Reformat. Fix bugs of char*
and QString[] being out of sync by eliminating one source of truth.

Fix crash in -^ generating doc list

vecs_t is no longer a struct of POD. It’s now a class and has to be
allocated by new. I hope to not have to rathole in vets for this change
because it’s full of scary memory management like this.

Resync with work copy

Merge branch 'master' into cleanups

xcdv to data structures continues

Passes testo. Exposes lots of weird stuff (all marked with FIXMEs) that
are still held up with C pointer magic, but we eliminate a few hundred
cases, so I’m just looking for a test run before digging deeper.

More data structure work in xcsv

Push QStringification up two levels. Keep table and callers of
substations in Qt-land to keep QHash happy and reduce copy/leak in most
cases.

fix codacy/cppcheck whining.

incorporate review suggestions and fix bugs.

correct user comment encoding based on image endianness.
fix possible tag corruption when data fits in 4 bytes depending
on image endianness and processor endianness.

Allow field options in ifield, ofield

Options can make fields.size() of 4, so loosen error test for those.
Misc warning fixes.

Pick up dropped file from last commit

Ooops. Add xcd.cc back to this CL.

Move csv & style handlers to real C++ containers

Goal: eliminate lots of manual string bashing.

Currently waffles between the two approaches, but I wanted to get this
into GitHub for valgrind results.

fix new exif memory leaks.

fix exif test to run in any time zone.

replace exif format usage of queues.

update gpx handling of global/metadata. (#236)

* update gpx handling of global/metadata.

use qt containers instead of queue.
add support for gpx/metadata/link.

* incorporate suggestions from review.

* update notes on justification for concatenating metadata.

Merge pull request #234 from tsteven4/reinterp

use reinterpret_cast with queue processing

use reinterpret_cast with queues.

use reinterpret_cast with queue processing

instead of a static cast to a void pointer followed
by a static cast to a Waypoint*.

Merge pull request #231 from tsteven4/jeeps

obsolete jeeps/gpsproj.h.

clean up some Variable * is reassigned a value before the old one has been used. (#232)

one of these was actually a very old bug.

obsolete jeeps/gpsproj.h.

Note that jeeps/gpsproj.cc is not used.
Historically some routines have been copied from gpsproj.cc to
gpsmath.cc.  When/If this is done the corresponding headers from
gpsproj.h should be copied to gpsmath.h.

allow unicsv bng input to use all numeric grid references.

const qualify some function parameters. (#229)

This is the clang-tidy check readability-non-const-parameter.

refactor "Declarator and assignment can be joined." (#227)

* refactor "Delarator and assignment can be joined."

Instances involving switch statements or goto statemens were skipped.
jeeps was skipped as well.

* fix pointer formatting case.

* more whitespace corrections.